Skip to content

[ci] Use static macOS pool for nightly emulator tests - #8206

Merged
pjcollins merged 1 commit into
dotnet:mainfrom
pjcollins:static-macos-nightly
Aug 3, 2023
Merged

[ci] Use static macOS pool for nightly emulator tests#8206
pjcollins merged 1 commit into
dotnet:mainfrom
pjcollins:static-macos-nightly

Conversation

@pjcollins

Copy link
Copy Markdown
Member

Attempts to improve the reliability and performance of our nightly
emulator tests by using a machine pool containing static mac minis that
are more performant than the hosted VMs.

@pjcollins
pjcollins force-pushed the static-macos-nightly branch from 6b119a1 to 74a59dc Compare July 20, 2023 19:54
@pjcollins

Copy link
Copy Markdown
Member Author

@pjcollins
pjcollins force-pushed the static-macos-nightly branch 2 times, most recently from f3b39b3 to 9041b46 Compare July 21, 2023 19:20
@pjcollins

Copy link
Copy Markdown
Member Author

The few test runs I've done here definitely seem more reliable, though the localization test category is still pretty flaky. My intuition is leaning towards the fact that trying to "soft restart" the emulator ~80 times per test run - https://github.com/xamarin/xamarin-android/blob/69b4ab056badcb5d25a5faf39d08baeb1b8b7010/tests/MSBuildDeviceIntegration/Tests/LocalizationTests.cs#L230 increases the likelihood of us eventually losing the connection to the emulator. I'm going to see if there are any pools with attached Android devices that we may be able to test with.

Attempts to improve the reliability and performance of our nightly
emulator tests by using a machine pool containing static mac minis that
are more performant than the hosted VMs.
@pjcollins
pjcollins force-pushed the static-macos-nightly branch from 9041b46 to a8aeb8c Compare August 1, 2023 17:53
@pjcollins

Copy link
Copy Markdown
Member Author

It doesn't seem any agents in the shared pool have an Android device demand defined, this may be the best we can do for now.

@pjcollins

Copy link
Copy Markdown
Member Author

@pjcollins
pjcollins marked this pull request as ready for review August 2, 2023 16:20
@pjcollins
pjcollins requested a review from jonpryor as a code owner August 2, 2023 16:20
@pjcollins
pjcollins requested review from dellis1972 and jpobst August 2, 2023 16:20

@jpobst jpobst left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to be a large pool of real mac machines. Can we also look at using them for our PR build tests?

@pjcollins

pjcollins commented Aug 3, 2023

Copy link
Copy Markdown
Member Author

This seems to be a large pool of real mac machines. Can we also look at using them for our PR build tests?

I have been looking at that in #8192. The owners would prefer we use the arm64 machines for PR builds and higher volume jobs, but I've been having performance issues there that I am trying to better understand. It's surprising because the arm64 pool should have m1 and m2 machines and generally be running better...

@pjcollins
pjcollins merged commit c22b2a8 into dotnet:main Aug 3, 2023
@pjcollins
pjcollins deleted the static-macos-nightly branch August 3, 2023 15:01
@github-actions github-actions Bot locked and limited conversation to collaborators Jan 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants